Skip to content

Document Mock client #70

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 7, 2016
Merged

Document Mock client #70

merged 1 commit into from
Jan 7, 2016

Conversation

ddeboer
Copy link
Contributor

@ddeboer ddeboer commented Jan 4, 2016

No description provided.

Further reading
---------------

* Read :ref:`message-factory` to learn how you can create request objects.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would that make sense in tests? i would think not, but if it does we should explain a little bit why you would want that.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Replaced this with the generic footer.

@dbu
Copy link
Contributor

dbu commented Jan 5, 2016

👍

i wonder if people get confused by the examples. we don't really test anything - maybe we should have a simple service that we want to test, to make the example more real life. like having an ApiClient that takes a Httplug client and parses json or something, then test that.

@sagikazarmark
Copy link
Member

👍 for examples. I think we are gonna need lots of examples. Example for an application, example for a library, example for the migration path, etc.

@ddeboer ddeboer force-pushed the mock-client branch 4 times, most recently from 7f97a62 to 470e497 Compare January 6, 2016 20:36
dbu added a commit that referenced this pull request Jan 7, 2016
@dbu dbu merged commit 237bf9a into master Jan 7, 2016
@dbu dbu deleted the mock-client branch January 7, 2016 07:14
@dbu
Copy link
Contributor

dbu commented Jan 7, 2016

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants